Skip to content

Feat metadata mapping#63

Open
micahwoodard wants to merge 58 commits into
mainfrom
feat-metadata-mapping
Open

Feat metadata mapping#63
micahwoodard wants to merge 58 commits into
mainfrom
feat-metadata-mapping

Conversation

@micahwoodard

@micahwoodard micahwoodard commented Apr 13, 2026

Copy link
Copy Markdown
Collaborator

Adds metadata mapping for acquisition, instrument, and data description as well as a cli. Based of vr foraging metadata mappers. Also some fixes for coupled trial generator. There are some things in data description like funding and investigators that need to be worked out

Comment thread src/aind_behavior_dynamic_foraging/data_contract/_dataset.py
Optional[float]: Total volume of water consumed in milliliters, or None if unavailable.
"""

trial_outcomes = dataset(session_path)["Behavior"]["SoftwareEvents"]["TrialOutcome"].load().data["data"]

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can validate this with the TrialOutcome pydantic class to make your life easier

Comment thread src/Extensions/bonsai.py
Comment thread workspace/aind_behavior_dynamic_foraging_metadata_mapper/pyproject.toml Outdated
@micahwoodard

Copy link
Copy Markdown
Collaborator Author

@arielleleon @saskiad this is the pr mapping the dynamic foraging output into an acquisition and instrument model.

@micahwoodard

Copy link
Copy Markdown
Collaborator Author

Updates mapper to classes to match vr foraging. resolves #97

@micahwoodard

Copy link
Copy Markdown
Collaborator Author

acquisition.json
instrument.json

@alexpiet these are new files generated with fixes for issues found in #107. There a couple of things I noted in the issue that will take more work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants